Skip to content

refactor(core): use atob and btoa instead of Buffer#72

Open
andrascodes wants to merge 1 commit intocore_expose_stampersfrom
andras/refactor-core-use-atob-and-btoa-instead-of-buffer
Open

refactor(core): use atob and btoa instead of Buffer#72
andrascodes wants to merge 1 commit intocore_expose_stampersfrom
andras/refactor-core-use-atob-and-btoa-instead-of-buffer

Conversation

@andrascodes
Copy link
Copy Markdown

@andrascodes andrascodes commented Apr 1, 2026

@andrascodes andrascodes force-pushed the andras/refactor-core-use-atob-and-btoa-instead-of-buffer branch from ebacb9a to da2f27c Compare April 1, 2026 17:11
@andrascodes andrascodes force-pushed the core_expose_stampers branch from 54fa0c4 to 1800c58 Compare April 1, 2026 17:11
@andrascodes andrascodes force-pushed the andras/refactor-core-use-atob-and-btoa-instead-of-buffer branch from da2f27c to f3577ac Compare April 1, 2026 21:08
@andrascodes andrascodes force-pushed the core_expose_stampers branch from 1800c58 to 7c1df70 Compare April 7, 2026 11:21
@andrascodes andrascodes force-pushed the andras/refactor-core-use-atob-and-btoa-instead-of-buffer branch 2 times, most recently from 4a78517 to 068598d Compare April 7, 2026 16:17
@andrascodes andrascodes marked this pull request as ready for review April 7, 2026 16:54
Copy link
Copy Markdown
Member

@OmarBasem OmarBasem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add context on the PR description on why we are removing Buffer, is it to make it work with RN?

@andrascodes
Copy link
Copy Markdown
Author

Can you add context on the PR description on why we are removing Buffer, is it to make it work with RN?

Yes. No Buffer in Hermes. Do you have a better alternative solution for this? I'd be happy to change it.

@andrascodes andrascodes force-pushed the andras/refactor-core-use-atob-and-btoa-instead-of-buffer branch from 068598d to c1d3e47 Compare April 20, 2026 10:11
@andrascodes andrascodes force-pushed the core_expose_stampers branch from 892cc8f to 3819862 Compare April 20, 2026 10:11
@andrascodes andrascodes force-pushed the core_expose_stampers branch from 3819862 to f3b5f87 Compare April 24, 2026 17:57
@andrascodes andrascodes force-pushed the andras/refactor-core-use-atob-and-btoa-instead-of-buffer branch from c1d3e47 to 746cf47 Compare April 24, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants